Skip to content

Conversation

@sangoi-exe
Copy link
Contributor

I was missing an example config.toml, and following docs/config.md alone was slower. I had GPT-5 scan the codebase for every accepted config key, check the defaults, and generate a single example config.toml with annotations. It lists all keys Codex reads from TOML, sets each to its effective default where it exists, leaves optional ones commented, and adds short comments on purpose and valid values. This should make onboarding faster and reduce configuration errors. I can rename it to config.example.toml or move it under docs/ if you prefer.

…one was slower. I had GPT-5 scan the codebase for every accepted config key, check the defaults, and generate a single example config.toml with annotations. It lists all keys Codex reads from TOML, sets each to its effective default where it exists, leaves optional ones commented, and adds short comments on purpose and valid values. This should make onboarding faster and reduce configuration errors. I can rename it to config.example.toml or move it under docs/ if you prefer.
Copy link
Collaborator

@etraut-openai etraut-openai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a link to this in the main README.md document. I think it makes sense for this to be a sub-bullet under "Configuration". We can call the sub-bullet "Example config" or something similar.

docs/config.toml Outdated
@@ -0,0 +1,301 @@
# Codex configuration template (config.toml)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's rename this file to docs/example-config.md so it can be displayed as a docs file.

docs/config.toml Outdated
@@ -0,0 +1,301 @@
# Codex configuration template (config.toml)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than referring to this as a "configuration template", let's call it an "example configuration" or something like that. I'm concerned that some users will find the term "template" to be confusing.

docs/config.toml Outdated
#
# This file lists all keys Codex reads from config.toml, their default values,
# and concise explanations. Values here mirror the effective defaults compiled
# into the CLI as of this repository. Adjust as needed.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"as of this repository" is a bit cumbersome. Maybe just delete this phrase.

@etraut-openai
Copy link
Collaborator

Thanks for the contribution, and apologies that it has taken so long to respond. Our PR backlog got quite long, and we're catching up.

I think this would be a good addition to the codex docs.

@sangoi-exe
Copy link
Contributor Author

Thanks for the contribution, and apologies that it has taken so long to respond. Our PR backlog got quite long, and we're catching up.

I think this would be a good addition to the codex docs.

Sure! No problem!

I just made the adjustments and also updated the file to match the keys that changed in the most recent releases.

Thanks for the reply!

@etraut-openai etraut-openai merged commit ab63a47 into openai:main Nov 4, 2025
10 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2025
@etraut-openai
Copy link
Collaborator

Thanks again for the contribution!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants